-
Notifications
You must be signed in to change notification settings - Fork 171
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support global-workflow using Rocky 8 on CSPs #2998
base: develop
Are you sure you want to change the base?
Conversation
…oud into csps-rocky8
…oud into csps-rocky8
…oud into csps-rocky8
Show tasks run on different partitions: compute and process
|
This UFS model hash is before the one that introduced the issue with building wave pre/post executables (see #3110), so it should be safe to update. |
This comment was marked as resolved.
This comment was marked as resolved.
Co-authored-by: David Huber <[email protected]>
…oud into csps-rocky8
…oud into csps-rocky8
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look good. Just a couple of questions.
Once the questions are addressed and reviews obtained from @DavidHuber-NOAA @WalterKolczynski-NOAA, we need to run this PR on Hera, Hercules, and WCOSS2 as this PR updates the submodules for UFS weather model, ufs-utils and gfs-utils. The AWSPW.env file also needs to remove the DA sections as that is not yet functional on the CSPs. @weihuang-jedi When you have addressed the comments, please let @DavidHuber-NOAA @WalterKolczynski-NOAA know so they can review and run the CI on the on-prem HPCs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved pending successful CI testing.
Description
With ParallelWorks now default Rocky 8 on CSPs, and move to Rocky 8 only after 1/1/2025,
we need to modify global-workflow module files to use Rocky 8 supported spack-stack,
and test compile and run to make sure all works under Rocky 8.
i) Rocky 8 update new features:
a. Wave worked in C48_S2SWA_gefs case, so turn SUPPORT_WAVES to "YES" in awspw.yaml.
Actually, if we did not set SUPPORT_WAVES to "YES", setup_expt.py will rise exception.
b. Using two type of nodes (chips/queues) on AWS, compute/process, where forecasts run in "compute" queue,
which is a big node (more cores), others run in "process" queue, which has small node (less cores).
ii) Rocky 8 update needs the following submodules at or newer than the tags below.
commit 4848ecbb5e713b16127433e11f7d3edc6ac784c4 (HEAD, origin/develop, origin/HEAD, develop)
Author: Wei Huang [email protected]
Date: Fri Oct 18 10:41:25 2024 -0600
commit 23237610845c3a4438b21b25e9b3dc25c4c15b73 (HEAD)
Author: Wei Huang [email protected]
Date: Wed Oct 9 11:55:13 2024 -0600
commit 66a422db80ea129dd87285fe6e811d4b6e1fe29b (HEAD)
Author: Wei Huang [email protected]
Date: Wed Oct 2 14:38:22 2024 -0600
commit 29c2703c715ebdb47bbd4bcc811db340eae530e5 (HEAD)
Author: Cameron Book [email protected]
Date: Tue Nov 12 13:08:12 2024 -0800
Resolves #2997
Type of change
Change characteristics
Does this change require a documentation update? YES/NO
How has this been tested?
Checklist